feat(firecrawl): add parse operation and revert short-input selection style#4340
feat(firecrawl): add parse operation and revert short-input selection style#4340waleedlatif1 merged 4 commits intostagingfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Updates the Firecrawl block/UI and tool registry to expose the new operation and its parse-specific options (formats, tag filters, parsers, timeout, ads/proxy/zero-retention), and updates docs/integration metadata accordingly. Also reverts the Reviewed by Cursor Bugbot for commit 29580b3. Configure here. |
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
Greptile SummaryThis PR adds a Confidence Score: 5/5Safe to merge — all findings are P2 style suggestions with no blocking defects. The new parse operation follows all established patterns for file-handling, auth, error forwarding, tool registration, and response transformation. No P0 or P1 issues found. Two P2 items: an accidental one-line removal in notion.mdx and a proxy dropdown missing a none entry. apps/docs/content/docs/en/tools/notion.mdx — verify the description removal is intentional. Important Files Changed
Sequence DiagramsequenceDiagram
participant User as User / Workflow
participant Block as FirecrawlBlock (parse case)
participant Tool as firecrawl_parse tool
participant Route as /api/tools/firecrawl/parse
participant Storage as File Storage
participant FC as Firecrawl API (/v2/parse)
User->>Block: params.document (file-upload or file-reference)
Block->>Block: normalizeFileInput(params.document)
Block->>Tool: { file, formats, onlyMainContent, options... }
Tool->>Route: POST JSON { apiKey, file, options }
Route->>Route: checkInternalAuth
Route->>Route: FirecrawlParseSchema.parse(body)
Route->>Route: processFilesToUserFiles(file)
Route->>Storage: downloadFileFromStorage(userFile)
Storage-->>Route: buffer
Route->>Route: new Blob(buffer) + FormData
Route->>FC: POST multipart/form-data { file, options }
FC-->>Route: { success, data: { markdown, ... } }
Route-->>Tool: { success: true, output: data }
Tool->>Tool: transformResponse → extract markdown, metadata, etc.
Tool-->>User: { markdown, summary, html, links, metadata, warning }
Reviews (2): Last reviewed commit: "fix(firecrawl): forward firecrawl error ..." | Re-trigger Greptile |
b0990fe to
5bb89f4
Compare
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 29580b3. Configure here.
… style (#4340) * feat(firecrawl): add parse operation and revert short-input selection style * chore(firecrawl): regenerate docs and integrations data for parse * fix(firecrawl): forward firecrawl error body in parse route response * fix(firecrawl): add pricing config to parse tool hosting
Summary
/v2/parseoperation to the Firecrawl block with file upload support (basic) and file reference (advanced)/api/tools/firecrawl/parsethat handles multipart/form-data upload to Firecrawlselection:text-transparentchange from fix(short-input): hide selected text to prevent overlay collision #4318 on short-input — did not work as intendedType of Change
Testing
Tested manually
Checklist